Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix auth when serving Frigate at a subpath #12815

Merged
merged 1 commit into from
Aug 9, 2024

Conversation

sorenisanerd
Copy link
Contributor

Ensure axios.defaults.baseURL is set when accessing login form.

Drop /api prefix in login form's axios.post call, since /api is part of the baseURL.

Redirect to subpath on succesful authentication.

Prepend subpath to default logout url.

Fixes #12814

Ensure axios.defaults.baseURL is set when accessing login form.

Drop `/api` prefix in login form's `axios.post` call, since `/api` is
part of the baseURL.

Redirect to subpath on succesful authentication.

Prepend subpath to default logout url.

Fixes blakeblackshear#12814
Copy link

netlify bot commented Aug 7, 2024

Deploy Preview for frigate-docs ready!

Name Link
🔨 Latest commit a996faa
🔍 Latest deploy log https://app.netlify.com/sites/frigate-docs/deploys/66b3c7ac12c486000873d4b1
😎 Deploy Preview https://deploy-preview-12815--frigate-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@NickM-27 NickM-27 changed the base branch from dev to 0.14.1 August 7, 2024 19:16
@NickM-27 NickM-27 deleted the branch blakeblackshear:dev August 8, 2024 13:54
@NickM-27 NickM-27 closed this Aug 8, 2024
@NickM-27 NickM-27 reopened this Aug 8, 2024
@NickM-27 NickM-27 changed the base branch from 0.14.1 to dev August 8, 2024 17:23
@NickM-27 NickM-27 merged commit c84511d into blakeblackshear:dev Aug 9, 2024
16 checks passed
hawkeye217 pushed a commit that referenced this pull request Aug 9, 2024
Ensure axios.defaults.baseURL is set when accessing login form.

Drop `/api` prefix in login form's `axios.post` call, since `/api` is
part of the baseURL.

Redirect to subpath on succesful authentication.

Prepend subpath to default logout url.

Fixes #12814
sandnabba pushed a commit to sandnabba/frigate that referenced this pull request Aug 14, 2024
Ensure axios.defaults.baseURL is set when accessing login form.

Drop `/api` prefix in login form's `axios.post` call, since `/api` is
part of the baseURL.

Redirect to subpath on succesful authentication.

Prepend subpath to default logout url.

Fixes blakeblackshear#12814
NickM-27 pushed a commit that referenced this pull request Aug 30, 2024
Ensure axios.defaults.baseURL is set when accessing login form.

Drop `/api` prefix in login form's `axios.post` call, since `/api` is
part of the baseURL.

Redirect to subpath on succesful authentication.

Prepend subpath to default logout url.

Fixes #12814
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants